Fix iteration past end in LevelDBTransaction::Iterator

Change Next() to avoid advancing past the end of either iterator by only
advancing if !advanced and if the iterator is still valid.
If Next() is called after the last entry is deleted, SyncToTransaction()
returns false indicating it hasn't advanced but at that point is_valid_
is false. Previously Next() would then attempt to advance one of the
underlying iterators and if it was the mutations_iter_ would advance
past the end, which would cause the next UpdateCurrent() to dereference
random memory.

An alternative fix would be to have SyncToTransaction return true but this changes the meaning of the return value from "advanced" to something like "caller shouldn't advance" but this didn't seem like a clear semantic.

Yeah while trying to verify my API changes were working I was running the full test battery and I saw this die with a SIGSEGV. The stack at that point indicated we were somehow past the end.

@wilhuff Note that this issue was also reported by ASan (which can be run from XCode).

Ah, got it. I'll probably need to figure out how to run more of those verifiers as I start writing more C++.

@gsoltis I want to write it up at some point, but basically, in XCode it's done by editing the current scheme: Product > Scheme > Edit Scheme, then select the applicable configuration ("Test" in this case) on the left, and check e.g. "Address Sanitizer" on the Diagnostics tab. (see Apple docs)


Where add Sourcetrail?

Free and open-source cross-platform source explorer
https://www.sourcetrail.com/

The project is dead. I suggest to close this issue.


Neo4j serializer

given the already graph like data model from apis and the existing neo4j python clients (https://neo4j.com/developer/python/) it should be rather easy to serialize apis data into a neo4j db

done, see https://github.com/acdh-oeaw/acdh-apis-neo4j


Update visual similarity stage in-place when possible in App workflows

Solves #1223
What changes are proposed in this pull request?
Visual similarity now updates the stage if the k value is unchanged from the previous one.
How is this patch tested? If it is not, please explain why.
Quick test:
import fiftyone as fo
import fiftyone.brain as fob
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset("quickstart")
fob.compute_similarity(dataset, brain_key="image_sim")

session = fo.launch_app(dataset)

query_id = dataset.first().id
stage = fo.SortBySimilarity(query_id)
view = dataset.add_stage(stage)

query_id = dataset.last().id
stage = fo.SortBySimilarity(query_id)
view = view.add_stage(stage)

session.view = view
session.wait()

Before:

After:

Additionally, you can try the click image + sort by similarity UI workflow.
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[x] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[ ] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[x] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Hi @cavesdev, I like what you did here, but we are looking for an App-side solution (TypeScript). I can point you in the right direction if that is in your wheelhouse.

I see that stages are mainly handled in the backend, that is why I implemented the function there.
Is there a function that updates the stage instead of adding a new one?
As I understand, this function is the one that adds a new stage.
https://github.com/voxel51/fiftyone/blob/06975961f5ee649dd36429feb21c959dfc0744ed/app/packages/app/src/components/Actions/Similar.tsx#L75-L89

@benjaminpkane can you help @cavesdev figure out the App-side implementation per his question?
@cavesdev thanks for taking this issue on! Like Ben mentioned, we think an App-side implementation makes the most sense here because, for API-based usage, we think it's best to avoid any "magic" optimizations and instead allow users to chain arbitrary view stages together, even if, strictly speaking, certain stages are nullified by future stages.
The duplicate SortBySimilarity stages from #1223 is strictly a consequence of the point-and-click workflows in the UI, so an App-side resolution seems appropriate.

Sorry, I missed this. No, there is not a function that updates a stage.
In the hook you referenced (within useRecoilCallback) you will have to interact with the view selector.
const params = await snapshot.getPromise(sortBySimilarityParameters);
const queryIds = await getQueryIds(snapshot, params.brainKey);
const view = await snapshot.getPromise(selectors.view);
set(similaritySorting, true);
set(atoms.modal, null);

// `view` is an array of serialized stages (readonly).
// Copy and modify for the new sorting (adding or editing a sort by stage)

set(selectors.view

I'm not totally on board with the idea of inspecting and modifying the view indirectly (outside the view bar). I'd rather see sorting by similarity moved to the extended view in some way. Anyways, this is fine with me for now.

I'm not totally on board with the idea of inspecting and modifying the view indirectly (outside the view bar). I'd rather see sorting by similarity moved to the extended view in some way. Anyways, this is fine with me for now.

Yeah I agree that similarity searches in the App shouldn't immediately be saved to the view bar. The extended view is the right place for it. Moving the similarity UI to the sidebar will take a bit of work though.
Is there a clean way to move the logic into the extended view but just keep the "state" of any active similarity search saved in the similarity menu itself? If the user executes a new similarity search, it would strictly overwrite the previous one (unless they first bookmark the extended view). The only requirement I see is that there be a clear visual indicator so that the user knows when they have a similarity search active that they might want to clear. I guess simply keeping the similarity icon visible even when no samples are selected could be enough.

@cavesdev Noting the above, to separate the similarity sorting from the current view, you'll need some knowledge of recoil.
Specifically, you will want to create a new atom that contains the sorting parameters. It can be defined Similar.tsx. Then you will include that atom as a part of page requests (defined in Flashlight.tsx and on the backend receive the new paramater(s) in main.py.
Because we are only sorting, only page queries need to be aware of the setting.
There are obviously more details, UX, etc. But if you are able to get sorting to work outside the view bar, we can then add pinning the sorting with filters and other details.

Sorry for the late reply, new year was around the corner. Thanks for the pointers (and the opportunity to work on this issue!), working on it!


Windows 11 with 1 external monitor "snap left" changes screens. Apparently it should not.

On windows 11 with a laptop monitor of 3k, and a 32" external monitor of similar.
With mostly stock knausj
If a window is fullscreen on my external saying "snap left" takes the window to my laptop screen. I struggle to get it back to external monitor.
@Andreas Arvidsson has seen it and confirmed it was weird.
I've not tried to debug it at all yet.
Once it was on the laptop screen something like "snap screen 1" gave me half a hugely scaled window on the external, and a normally scaled on the laptop.
The "snap screen " seem weird, too.

If you could take a screenshot of the display preferences that shows your screen layout that would make it easier to reproduce. Thanks!


Include Temperature Sensors

I have an AEG RME954F9VX french door refrigerator and thought I would give it a try. Some sensors actually work - thanks for that! Great work!
I receive: Air filter lifetime and -state, water filter flow (I think this is in liters), water filter life time and -state, defrost routine state, humidity, and main board sw version, as well as WIFI connectivity state and link quality indicator.
I wanted to ask if it is possible to add more sensors. I would be interested in three temperature sensors that can be seen in the AEG (Electrolux) app: Refrigerator temperature, the temperature of the first drawer and the freezer compartment temperature.

Hi,
normally all the available sensors are added. If you have doubts, there is a python script provided by the integration
https://github.com/albaintor/homeassistant_electrolux_status/blob/main/testAppliance.py
You can try it (after installing python on your system) after having modified the credentials in the script.
You should be able to see all the available features/fields reported by your appliance

Hi! Thank you very much! I also turned on debug mode and saw some values that are not available as a sensor. How can I turn them into sensors? As far as i could see, it is the data after " 'fridge': " , " 'iceMaker': " and " 'frezer': "
2024-05-03 05:46:53.982 DEBUG (MainThread) [custom_components.electrolux_status] Electrolux updated reported data {'applianceId': '###', 'applianceData': {'applianceName': '###', 'created': '2024-04-22T20:03:28.796Z', 'modelName': 'CR'}, 'properties': {'desired': {}, 'reported': {'defrostRoutineState': 'WAIT', 'applianceInfo': {'applianceType': 'CR'}, 'fridge': {'alerts': [], 'doorState': 'CLOSED', 'fastModeTimeToEnd': 0, 'fastMode': 'OFF', 'targetTemperatureF': 37.4, 'targetTemperatureC': 3.0, 'applianceState': 'RUNNING'}, 'ui2LockMode': False, 'uiLockMode': False, 'sabbathMode': 'OFF', 'extraCavity': {'fanState': 0}, 'vacationHolidayMode': 'OFF', 'applianceMode': 'NORMAL', 'applianceMainBoardSwVersion': 'JHB10V00', 'alerts': [], 'iceMaker': {'defrostTemperatureC': -20.5, 'defrostTemperatureF': -4.899999999999999}, 'waterFilterState': 'GOOD', 'waterFilterLifeTime': 895299, 'waterFilterFlow': 39.0, 'networkInterface': {'swVersion': 'v5.4.2', 'linkQualityIndicator': 'GOOD', 'otaState': 'IDLE', 'niuSwUpdateCurrentDescription': 'A07491702B-S00008607A', 'swAncAndRevision': 'S00008607A'}, 'airFilterLifeTime': 895299, 'sensorHumidity': 47, 'freezer': {'alerts': [], 'doorState': 'CLOSED', 'fastModeTimeToEnd': 0, 'fastMode': 'OFF', 'targetTemperatureF': -0.3999999999999986, 'targetTemperatureC': -18.0, 'applianceState': 'RUNNING'}, 'airFilterState': 'GOOD', 'reminderTime': -1, 'connectivityState': 'connected'}, 'metadata': {}}, 'status': 'enabled', 'connectionState': 'connected'}

I found a solution: I changed line 39 in const.py
from COMMON_ATTRIBUTES = ["connectivityState", "networkInterface/linkQualityIndicator", "applianceMode"]
to COMMON_ATTRIBUTES = ["connectivityState", "networkInterface/linkQualityIndicator", "applianceMode", "fridge/targetTemperatureC", "iceMaker/defrostTemperatureC", "freezer/targetTemperatureC", "fridge/doorState", "freezer/doorState"]
works now for me & can be closed


cannot access the log file

I am trying my first case (not sample patient) and got a few times the same error from the executable:
The process cannot access the log file (patientID.txt) because it is being used by another process.
That last time it started the optimization at 6:15am, after 3rd iterations it was able to write to the log file at 10:03am, then it complains that the file is not accessible at sometime before 10:20am. Pretty sure no one is touching the log file.
The only thing I can think of is maybe it's expecting to access the file in some milliseconds but did not get response quick enough? My files are in a network drive with a bit long path. Thanks.

Interesting. I haven't seen this error come up before (other than when someone is explicitly messing with the log file). Does it throw this error when planning the sample cases? If not, that's interesting. If yes, then it might be because of your theory. A simple check would be to change the log file path to a local directory on the computer running the optimization loop. If you don't get the error, then it might be due to trying to access the network drive. If you do get the error on a local drive, we'll have to dig a bit deeper to find the error. To prevent the script from crashing, replace line 687 in VMATTBI_optLoopMT/progressWindow.xaml.cs with:
try
{
File.AppendAllText(fileName, output);
}
catch(Exception e)
{
provideUpdate("Error! Cannot write to log file!");
provideUpdate(e.Message);
provideUpdate(e.StackTrace);
}
Then recompile and try again. If the script can't write to the file it will print a big long error message to the progress window, but it should keep going. Let me know what you find.

I copied all the files to a local C drive, so far the error has not coming up yet. While a day ago it appeared at least 2-3 times. I assume it is because of accessing the network drive files and our hospital network often has hiccups.  I assume it is solved, if not, I will report it here again. Thank you for very prompt answering.

Sounds good. Keep me posted on what happens. I'll leave this issue open until you confirm it was a network drive issue

Assuming this issue is resolved. Closing


make data_feeder for sparse matrices

Environment info
Operating System: OS X El Capitan 10.11.4
Installed version of CUDA and cuDNN: N/A

pip install: https://storage.googleapis.com/tensorflow/mac/tensorflow-0.8.0-py2-none-any.whl
version: 0.8.0

Steps to reproduce
Run this code with svm light data:
from sklearn import datasets, metrics
from tensorflow.contrib import skflow

train_data, train_labels = datasets.load_svmlight_file(TRAIN)

classifier = skflow.TensorFlowDNNClassifier(hidden_units=[10, 20, 10], n_classes=2)
classifier.fit(train_data, train_labels)
score = metrics.accuracy_score(train_labels, classifier.predict(train_data))

What have you tried?
I think the classifier doesn't handle the CSR matrix matrix produced by the loader
Logs or other output that would be helpful
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-27-01a3edd34655> in <module>()
      1 classifier = skflow.TensorFlowDNNClassifier(hidden_units=[10, 20, 10], n_classes=2)
----> 2 classifier.fit(train_data, train_labels)
      3 score = metrics.accuracy_score(train_labels, classifier.predict(train_data))

/Users/nicolas/anaconda/envs/tensorflow/lib/python2.7/site-packages/tensorflow/contrib/learn/python/learn/estimators/base.pyc in fit(self, X, y, monitor, logdir)
    216         self._data_feeder = setup_train_data_feeder(X, y,
    217                                                     self.n_classes,
--> 218                                                     self.batch_size)
    219
    220         if monitor is None:

/Users/nicolas/anaconda/envs/tensorflow/lib/python2.7/site-packages/tensorflow/contrib/learn/python/learn/io/data_feeder.pyc in setup_train_data_feeder(X, y, n_classes, batch_size)
     97                              "streaming learning to work.")
     98         data_feeder_cls = StreamingDataFeeder
---> 99     return data_feeder_cls(X, y, n_classes, batch_size)
    100
    101

/Users/nicolas/anaconda/envs/tensorflow/lib/python2.7/site-packages/tensorflow/contrib/learn/python/learn/io/data_feeder.pyc in __init__(self, X, y, n_classes, batch_size, random_state)
    189         x_dtype = np.int64 if X.dtype == np.int64 else np.float32
    190         y_dtype = np.int64 if n_classes > 1 else np.float32
--> 191         self.X = check_array(X, dtype=x_dtype)
    192         self.y = (None if y is None else check_array(y, dtype=y_dtype))
    193         self.n_classes = n_classes

/Users/nicolas/anaconda/envs/tensorflow/lib/python2.7/site-packages/tensorflow/contrib/learn/python/learn/io/data_feeder.pyc in check_array(array, dtype)
    159         Original array or converted.
    160     """
--> 161     array = np.array(array, dtype=dtype, order=None, copy=False)
    162     return array
    163

ValueError: setting an array element with a sequence.

Yes, currently it only support dense numpy matrix / pandas DataFrame and Series.
If you are interested in adding a data feeder for sparse matricies - that would be great contribution!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


New version: Polynomials v0.9.0

Registering package: Polynomials
Repository: https://github.com/JuliaMath/Polynomials.jl
Created by: @jverzani
Version: v0.9.0
Commit: 4ace8243b4531c58ccda5a682378ac91e5c053f0
Reviewed by: @jverzani
Reference: https://github.com/JuliaMath/Polynomials.jl/commit/4ace8243b4531c58ccda5a682378ac91e5c053f0#commitcomment-38586447

comment, please don't merge. Will make a 0.8 release


Added !default qualifier to color and size variables.

Referencing issue #7

Yep, this lgtm. Closes #7 - thanks!


added PARALLAX_WEBSITE

Title and Issue number
Title : PARALLAX_WEBSITE
Issue No. :#240
Code Stack : HTML ,CSS
Close #240
Video (mandatory)
https://www.loom.com/share/f0142a2283d54c508e6175b9554ba066
Checklist:

[ X] I have mentioned the issue number in my Pull Request.
[X ] I have commented my code, particularly in hard-to-understand areas
[X ] I have created a helpful and easy to understand README.md
[X ] I have updated the projectData.json file for my contribution
[X ] I have added the img of my project in the assets/img section

Additional context (Mandatory )
PARALLAX WEBSITE WITH PARALLAX EFFECT
Are you contributing under any Open-source programme?
CODEPEAK 23

@GAGANRAGHAV Great! Thanks for contributing.


Vendor extensions

I am trying to use this library to send 0x101B (GetPartialObject) request to MTP device and than read the data. I can't find the way to do that. What the inputParams should be? How to read the received data?
Thank you.

Hi Luka,
check the Vendor… methods in the MediaDevice class.
Some are commented out because I found no device to test it.
Feel free to test it and send me your changes.
Regards,
Ralf


Changes to auxin influx/efflux

Hi,
I am making these changes to have unique logical definitions for transport BP terms, see #22721

[x]  GO:0010315 auxin efflux

Change label to  'auxin export across the plasma membrane'

Change logical def to
intersection_of: GO:0006810 ! transport
intersection_of: has_input GOCHE:22676 ! substance with auxin role
+intersection_of: has_target_end_location GO:0005576 ! extracellular region
+intersection_of: has_target_start_location GO:0005622 ! intracellular anatomical structure
+intersection_of: results_in_transport_across GO:0005886 ! plasma membrane

[x]  GO:0060919 auxin influx

Change label to  'auxin import into cell'

Change logical def to
intersection_of: GO:0006810 ! transport
intersection_of: has_input GOCHE:22676 ! substance with auxin role
+intersection_of: has_target_end_location GO:0005622 ! intracellular anatomical structure
+intersection_of: has_target_start_location GO:0005576 ! extracellular region
+intersection_of: results_in_transport_across GO:0005886 ! plasma membrane

[x]  GO:0080162  intracellular auxin transport (single annotation)

Change label to : 'endoplasmic reticulum to cytosol auxin transport'

REMOVE is_a: GO:0060919 ! auxin influx

REMOVE intersection_of: GO:0060918 ! auxin transport

REMOVE intersection_of: occurs_in GO:0005622 ! intracellular anatomical structure

Change logical def to
+intersection_of: GO:0006810 ! transport
+intersection_of: has_input GOCHE:22676 ! substance with auxin role
+intersection_of: has_target_end_location GO:0005829 ! cytosol
+intersection_of: has_target_start_location GO:0005788 ! endoplasmic reticulum lumen
+intersection_of: results_in_transport_across GO:0005789 ! endoplasmic reticulum membrane

@tberardini I checked that this was consistent with existing annotations; let me know if you'd like me to make other modifications.
Thanks, Pascale

@pgaudet - Is the plan to eventually use 'has primary input' in place of 'has input' for these kinds of transport logical defs?
I'm also looking at https://github.com/geneontology/go-ontology/issues/22721

@pgaudet  I did not see this ticket before. :(  Please confirm that you retained the original primary term names as synonyms.  Thanks.

@tberardini this is weird, usually we get an error for not having the former term label as a synonym, but I thought all checked had passed! I now added the synonym back.
Is there anything else you'd like changed?

I can't remember how to view the final term block. If you would like me to have one more look, please copy/paste them and I'll do so.

Yeah the diff only shows partial information. Here are the changes for the 3 terms:
id: GO:0010315
name: auxin export across the plasma membrane
namespace: biological_process
def: "The directed movement of auxins from inside of a cell, across the plasma membrane and into the extracellular region." [GOC:tair_curators, PMID:16990790]
synonym: "auxin efflux" BROAD []
synonym: "auxin export" BROAD []
intersection_of: GO:0006810 ! transport
intersection_of: has_primary_input GOCHE:22676 ! substance with auxin role
intersection_of: has_target_end_location GO:0005576 ! extracellular region
intersection_of: has_target_start_location GO:0005737 ! cytoplasm
intersection_of: results_in_transport_across GO:0005886 ! plasma membrane
relationship: regulates GO:0009734 ! auxin-activated signaling pathway
id: GO:0060919
name: auxin import into cell
namespace: biological_process
def: "The directed movement of auxins from outside of a cell into a cell." [PMID:22773749]
synonym: "auxin influx" BROAD []
is_a: GO:0060918 ! auxin transport
intersection_of: GO:0006810 ! transport
intersection_of: has_primary_input GOCHE:22676 ! substance with auxin role
intersection_of: has_target_end_location GO:0005829 ! cytosol
intersection_of: has_target_start_location GO:0005576 ! extracellular region
intersection_of: results_in_transport_across GO:0005886 ! plasma membrane
id: GO:0080162
name: endoplasmic reticulum to cytosol auxin transport
namespace: biological_process
def: "The directed movement of auxins from endoplasmic reticulum to cytosol." [PMID:19506555]
synonym: "intracellular auxin transport" BROAD []
intersection_of: GO:0006810 ! transport
intersection_of: has_primary_input GOCHE:22676 ! substance with auxin role
intersection_of: has_target_end_location GO:0005829 ! cytosol
intersection_of: has_target_start_location GO:0005788 ! endoplasmic reticulum lumen
intersection_of: results_in_transport_across GO:0005789 ! endoplasmic reticulum membrane

Do you want to be consistent with cytosol/cytoplasm?  2/3 are cytosol. Otherwise, this looks good. Thanks for the additional details.


Support recurring events from .ics

This implementation will generate reccuring events based on RRULE in ics file. It will generate infinity events only for 1 year, so if you want to see more events in next years you need to reload data in that year (with Q)
note: please feel free to comment and repair me or offer a better solution, I am not python developer :innocent:
Closes #94

Thank you for working on this! I'll check and test as soon as I can.

Thank you again for your contribution! I seem to work fine. Ideally, I'd prefer all the rrule stuff to be read and converted to internal fields like repetitions and frequency already in the loaders.py, so that we don't carry those field through the program. However, I guess it's not always possible if the rrule is more complicated than "repeat every X for N times". So, let's keep it like that for now, as it works.

By the way, while we are at it, may I ask what is exactly exdate and what it contains?

Yep, my first try was exactly converting to internal fields but as you guessed, real events can be more complicated. You can have specific days for recurring event and you can have interval between them (each 2 weeks instead of each week). And exdate is for exclude specific date from repetition. (If you do remove on recurring event in google calendar and choose Only this event)

I see, okay, thank you for the explanation !


vim not getting build

Im trying to install vim with brew and im getting make error everytime that i try to install, i had no problems with other packages like python or gcc, i run brew doctor and i had no issue at all
https://gist.github.com/anonymous/235fcb55542a13100af2ce1ea4810da2
this is the console error
brew install vim
==> Downloading https://github.com/vim/vim/archive/v7.4.1755.tar.gz
Already downloaded: /home/schizoidcock/.cache/Homebrew/vim-7.4.1755.tar.gz
==> ./configure --prefix=/home/schizoidcock/.linuxbrew --mandir=/home/schizoidco
==> make
Last 15 lines from /home/schizoidcock/.cache/Homebrew/Logs/vim/02.make:
cd src && make first
make[1]: Entering directory '/tmp/vim-20160501-24468-1o5b2tj/vim-7.4.1755/src'
mkdir objects
CC="/usr/bin/gcc -Iproto -DHAVE_CONFIG_H   -isystem/home/schizoidcock/.linuxbrew/include    " srcdir=. sh ./osdef.sh
creating auto/pathdef.c
/usr/bin/perl -e 'unless ( $] >= 5.005 ) { for (qw(na defgv errgv)) { print "#define PL_$_ $_\n" }}' > auto/if_perl.c
/usr/bin/perl  -prototypes -typemap \
    /usr/share/perl5/ExtUtils/typemap if_perl.xs >> auto/if_perl.c
Unrecognized switch: -rototypes  (-h will show valid options).
Makefile:2659: recipe for target 'auto/if_perl.c' failed
make[1]: *** [auto/if_perl.c] Error 25
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory '/tmp/vim-20160501-24468-1o5b2tj/vim-7.4.1755/src'
Makefile:26: recipe for target 'first' failed
make: *** [first] Error 2

/usr/bin/perl  -prototypes -typemap \
    /usr/share/perl5/ExtUtils/typemap if_perl.xs >> auto/if_perl.c
Unrecognized switch: -rototypes  (-h will show valid options).

Please report /usr/bin/perl --version
Try brew install perl vim

It got this error with v5.18.2

Please open a new issue over at https://github.com/Linuxbrew/homebrew-core/issues


Make tagging and collection page more consistent

Describe what feature you'd like. Pseudo-code, mockups, or screenshots of similar solutions are encouraged!
Collection

Full width?
JPEG → Foto (JPEG,PNG...)
Tagging more consistent

What type of pull request would this be?
None
Any links to similar examples or other references we should review?
No response

[ ] Type (Compound et al) can be removed

is covered in #77


create flashcards for sentence structures

Automatically determine the sentence structure(s) required for a given sentence and create flashcards for these.

I think it's probably easier for users to use a well-curated grammar wiki, such as https://resources.allsetlearning.com/chinese/grammar/
they probably sell flashcards based around their structure points, and I don't think this feature would give good return on time invested.
@danielt998 You want us to implement this still, or?


Can I log my self message?

If not, can I modify this project to support it?

You can feel free to contribute to the project if you like, i will review and marge any pull request if its good

refloow.on("friendMessageEcho") could do this


Access Store from stronghold

Description of change
This PR gives session based access to the insecure Store of a running Stronghold instance.
Links to any relevant issues
#429
Type of change

[x] Enhancement (a non-breaking change which adds functionality)

How the change has been tested
Integration tests have been provided.
Change checklist

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Closing. Duplicate effort.


AttributeError: 'Field' object has no attribute 'vocab'

Which notebook/cell is giving you this error?


cmdlet does not work when  specifying the values of arguments when asked but works when specifying altogether with the cmdlet

I assigned the object ID's to 2 different variables and then provided it when asked. It didn't work.
when specifying the same values along with the cmdlet, it worked. How the execution is different in both the cases?
PS C:\Users\Ankit> $abhi=Get-AzureADUser -SearchString abhi
PS C:\Users\Ankit> Add-AzureADGroupMember
cmdlet Add-AzureADGroupMember at command pipeline position 1
Supply values for the following parameters:
ObjectId: $group.objectid
RefObjectId: $abhi.objectid
Add-AzureADGroupMember : Error occurred while executing AddGroupMember
Code: Request_BadRequest
Message: The request URI is not valid. Since the segment 'Microsoft.DirectoryServices.Group' refers to a collection, this must be the last
segment in the request URI. All intermediate segments must refer to a single resource.
RequestId: 108c5cb0-0af5-42d5-bd63-8bf3bbe746e4
DateTimeStamp: Fri, 19 Jun 2020 13:57:51 GMT
HttpStatusCode: BadRequest
HttpStatusDescription: Bad Request
HttpResponseStatus: Completed
At line:1 char:1

Add-AzureADGroupMember

  + CategoryInfo          : NotSpecified: (:) [Add-AzureADGroupMember], ApiException
  + FullyQualifiedErrorId : Microsoft.Open.AzureAD16.Client.ApiException,Microsoft.Open.AzureAD16.PowerShell.AddGroupMember

#THE BELOW CMDLET JUST WORKED FINE
PS C:\Users\Ankit> Add-AzureADGroupMember -ObjectId $group.ObjectId -RefObjectId $abhi.ObjectId

Hi @ankitjoshi8518
Thank you for your issue submission.  Unfortunately, this appears to be a problem with the cmdlt and a support issue.  Issues in this repo are for documentation issues.  You can try one of the following links for additional help.
Microsoft Support:  https://support.microsoft.com/
Azure AD forums: https://social.technet.microsoft.com/Forums/en-US/home?forum=windowsazureaditpro
Stackoverflow: https://stackoverflow.com/
#please-close


[IMPORTANT] Demo site is broken

@jakewies could you look into this?

I think the build process for that is somehow broken

This is a strange issue. The master branch builds just fine, but the hosted demo on themes.gohugo.io break. The hosted demo is also trying to request scss files instead of the static/css files.

@jakewies is the hosted demo using hugo-extended? also i think you linked the wrong issue in your comment

It could be that. I mentioned it in the issue. I also updated the issue link.

Oh I didn't know there was a repo for all the themes. It turns out:
Each theme needs:
5. To include the resources folder if it's using Hugo Pipes to process assets;

Looks like we need to include the /resources/_gen folder for the demo site to work. I've also looked at a theme that I used previously and they did that as well.

@jakewies can you update the commit reference in the themes repo too? I don't know how to do that

@kentnek Not sure how to do that either. If you're referring to the fact that the theme demo is still broken even after https://github.com/jakewies/hugo-theme-codex/pull/62, it might be because of this. Looks like the themes get updated at:
00:00 UTC on the 1, 4, 7, 10, 13, 16, 19, 22, 25, 28 and 31th of every month

The readme for the hugo themes repo doesn't have any docs on how to update a commit reference. I will go ahead and create a new release in GitHub for this theme in the meantime, and we'll keep an eye on the demo in the coming days.


Add missing FOSRest changes

I don't know when this package was created in relation to FOSRest, but there are some changes which are not applied here.
This is an example:
https://github.com/FriendsOfSymfony/FOSRestBundle/pull/2091/files
Is this PR the only difference or are there any other?
Thank you.

@mlazze  Thank you for your report. There can be other changes I was not aware that there where features added in 2.8 for the deprecated classes.
Do you want to provide a PR for your linked changes?

This was implemented in #16 by @kevinpapst.


clusterAffinity of Globalization not working

What happened:
I want to override different configs for a subscription  according to different labels of mcls, so I  created two Globalization and use matchLabels of clusterAffinity field to query different mcls , but clusterAffinity does  not  work, all overrides of Globalizations merged in target description
Environment:

Clusternet version:

Clusternet-agent version (user clusternet-agent --version=json):0.9.1
Clusternet-hub version (user clusternet-hub --version=json): 0.9.1

Kubernetes version (use kubectl version): k3s  1.22
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release): ubuntu 20.04

@jasine Could you please share your Globalization specifications?

@dixudx
spec:
  clusterAffinity:
    matchLabels:
      accelerator: npu
  feed:
    apiVersion: apps.clusternet.io/v1alpha1
    kind: HelmChart
    name: dipper
    namespace: dipper
  overridePolicy: ApplyNow
  overrides:
  - name: override-npu-config
    type: Helm
    value: |-
      accelerator:
        npu: true
      logLevel: debug
      imagePullPolicy: Always
      imagePullSecret:
        username: "xxxxx"
        password:  xxxxx
      .... # other configs
  priority: 600

For Globalization, the clusterAffinity is not implemented yet. And it will be probably marked as deprecated, since Feed is enough to match a resource explicitly.

@dixudx I think clusterAffinity of Globalization is still useful, for example, in my scenario, if  use Feed, I have to create  two feeds and  two Globalization to override them, while if clusterAffinity of Globalization works, one less resource (feed ) is created

while if clusterAffinity of Globalization works, one less resource (feed ) is created

@jasine Right, clusterAffinity will still be useful.
Would you like to implement this feature?

OK, let me have a try


Set up travis for running python unittests

We should add a flake8.cfg to the mix, there are a lot of small things like unused imports lying around.

If I can compile GDAL without problems I would be a volunteer to work on this.

Couple more thoughts:

Tests need to run against multiple version of python (at least 2.7 and 3.5), might make sense to use tox
Might make sense to use CircleCI instead (SSH access might be necessary with the in depth setup we have)

@dorukozturk I agree with @danlamanna completely.  :+1: for circleci :+1: for tox.  I know you've already put some work into getting travis up and running but I think dan makes a compelling point about ssh access and GDAL issues.  Looking through the circleci docs it looks like an all around better experience.

@kotfic should we close this?


[SPARK-11396] [SQL] add native implementation of datetime function to_unix_timestamp

to_unix_timestamp is the deterministic version of unix_timestamp, as it accepts at least one parameters.
Since the behavior here is quite similar to unix_timestamp, I think the dataframe API is not necessary here.

Merged build triggered.

Merged build started.

Test build #44565 has started for PR 9347 at commit 7c595fd.

Merged build triggered.

Merged build started.

Test build #44566 has started for PR 9347 at commit c684a7b.

Test build #44565 has finished for PR 9347 at commit 7c595fd.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression) extends UnixTime \n  * case class UnixTimestamp(timeExp: Expression, format: Expression) extends UnixTime \n  * abstract class UnixTime extends BinaryExpression with ExpectsInputTypes \n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44565/
Test PASSed.

Test build #44566 has finished for PR 9347 at commit c684a7b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression) extends UnixTime \n  * case class UnixTimestamp(timeExp: Expression, format: Expression) extends UnixTime \n  * abstract class UnixTime extends BinaryExpression with ExpectsInputTypes \n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44566/
Test PASSed.

@liancheng

LGTM overall, just one minor comment.

Merged build triggered.

Merged build started.

Test build #45509 has started for PR 9347 at commit e6201c5.

Test build #45509 has finished for PR 9347 at commit e6201c5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression)\n  * abstract class UnixTime(timeExp: Expression, format: Expression)\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45509/
Test FAILed.

retest this please.

Merged build triggered.

Merged build started.

Test build #45513 has started for PR 9347 at commit e6201c5.

Test build #45513 has finished for PR 9347 at commit e6201c5.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression)\n  * abstract class UnixTime(timeExp: Expression, format: Expression)\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45513/
Test FAILed.

Seems a legitimate failure (java.io.InvalidClassException: org.apache.spark.sql.catalyst.expressions.UnixTimestamp; no valid constructor).

Merged build triggered.

Merged build started.

Test build #45602 has started for PR 9347 at commit 533775f.

Test build #45602 has finished for PR 9347 at commit 533775f.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression)\n  * abstract class UnixTime(timeExp: Expression, format: Expression)\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45602/
Test FAILed.

Please fix the bug for SparkR
java.io.InvalidClassException: org.apache.spark.sql.catalyst.expressions.UnixTimestamp; no valid constructor
	at java.io.ObjectStreamClass$ExceptionInfo.newInvalidClassException(ObjectStreamClass.java:150)
	at java.io.ObjectStreamClass.checkDeserialize(ObjectStreamClass.java:768)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1772)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.readArray(ObjectInputStream.java:1706)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1344)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.readObject(ObjectInputStream.java:370)
	at scala.collection.immutable.$colon$colon.readObject(List.scala:362)
	at sun.reflect.GeneratedMethodAccessor5.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:606)
	at java.io.ObjectStreamClass.invokeReadObject(ObjectStreamClass.java:1017)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1893)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:1990)
	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:1915)
	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:1798)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1350)
	at java.io.ObjectInputStream.readObject(ObjectInputStream.java:370)
	at org.apache.spark.serializer.JavaDeserializationStream.readObject(JavaSerializer.scala:76)
	at org.apache.spark.serializer.JavaSerializerInstance.deserialize(JavaSerializer.scala:115)
	at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:61)
	at org.apache.spark.scheduler.Task.run(Task.scala:88)
	at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:214)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
	at java.lang.Thread.run(Thread.java:745)

Merged build triggered.

Merged build started.

Test build #45627 has started for PR 9347 at commit 1636f72.

Test build #45627 has finished for PR 9347 at commit 1636f72.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression)\n  * abstract class UnixTime(timeExp: Expression, format: Expression)\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45627/
Test FAILed.

retest this please.

Merged build triggered.

Merged build started.

Test build #45629 has started for PR 9347 at commit 1636f72.

Test build #45629 has finished for PR 9347 at commit 1636f72.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression)\n  * abstract class UnixTime(timeExp: Expression, format: Expression)\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45629/
Test FAILed.

Merged build triggered.

Merged build started.

default constructor has conflict here, so maybe we have to avoid parameters in abstract class UnixTime.

Test build #45646 has started for PR 9347 at commit b2fca64.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45646/
Test FAILed.

Test build #2041 has started for PR 9347 at commit b2fca64.

Test build #2041 has finished for PR 9347 at commit b2fca64.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression) extends UnixTime \n  * case class UnixTimestamp(timeExp: Expression, format: Expression) extends UnixTime \n  * abstract class UnixTime extends BinaryExpression with ExpectsInputTypes \n

retest this please.

Merged build triggered.

Merged build started.

Test build #45699 has started for PR 9347 at commit b2fca64.

Test build #45699 has finished for PR 9347 at commit b2fca64.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class ToUnixTimestamp(timeExp: Expression, format: Expression) extends UnixTime \n  * case class UnixTimestamp(timeExp: Expression, format: Expression) extends UnixTime \n  * abstract class UnixTime extends BinaryExpression with ExpectsInputTypes \n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45699/
Test PASSed.

LGTM, merging this into master and 1.6.


watch-storefront does not work anymore

Describe the bug
The watch-storefront makefile command does not work anymore since some recent version. we cannot exactly tell which version, since we used the latest image and the issue occured after a docker system prune -a today, which caused a new download of the image of course.
After some hours we found out that the .htaccess.watch file in the /html directory is no longer respected. this comes down to the fact that the apache config does not even involve this directory (anymore). moving the .htaccess.watch file to the public directory helps, but this overrides the .htaccess file then. the only real fix for us was to move the RequestHeader instructions from the .htacces.watch file in /html into the .htaccess file in /html/public
To Reproduce
Steps to reproduce the behavior:

Pull the latest image
Start the make watch-storefront command
Hot reload does not work

Expected behavior
Hot reload should work

Hi
thank you for this issue
we are already aware of, and its already fixed....about to be released with 1.3.1 soon :)
best regards
Christian

fixed in master, will be pushed to docker hub today


python runtests.py -v fails on master branch w/sha 79dae16990d6d17398a372cae22a291bf23de980

My issue is about the runtests.py file fails when run on mast recent commit.
Reproducing code example:

python runtests.py -v

Error message:

Building, see build.log...
Build OK (0:00:05.539618 elapsed)
================================================================================ test session starts ================================================================================
platform darwin -- Python 3.6.6, pytest-3.7.3, py-1.5.4, pluggy-0.7.1 -- /Users/rlucas/anaconda3/envs/scipy-dev/bin/python
cachedir: .pytest_cache
rootdir: /Users/rlucas/scipy-dev/scipy, inifile: pytest.ini
collected 16664 items / 1 errors / 1223 deselected

====================================================================================== ERRORS =======================================================================================
_________________________________ ERROR collecting build/testenv/lib/python3.6/site-packages/scipy/optimize/tests/test__linprog_ip_clean_inputs.py __________________________________
ImportError while importing test module '/Users/rlucas/scipy-dev/scipy/build/testenv/lib/python3.6/site-packages/scipy/optimize/tests/test__linprog_ip_clean_inputs.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
scipy/optimize/tests/test__linprog_ip_clean_inputs.py:9: in <module>
    from scipy.optimize._linprog_ip import _clean_inputs
E   ImportError: cannot import name '_clean_inputs'
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
===================================================================== 1223 deselected, 1 error in 11.46 seconds =====================================================================

Scipy/Numpy/Python version information:

I updated my local repo master branch with master from github/scipy (was about 130 commits behind prior to the last release). Working on getting things back to a useable state as before. The docs all build in sphinx successfully. I was also able to run (and pass) all tests in the stats submodule (where I've mostly been contributing. I also got passing tests on the following submodules [stats, special, spatial, sparse].  Also, 88 tests failed on the signal submodule e.g. python runtests.py -v -s signal.

Did you first run git clean -xdf and then build it from scratch with python setup.py build ?

I did not. Should I be doing a clean on every git pull from the scipy/scipy upstream?
That appears to fix the issue w/the test in optimize submodiule but I'm still getting all the failures in the signal submodule?

I'm not sure anymore because I can't keep up with the dependency updates hence I'm just doing it anyways. But that's out of ignorance. Someone might know better why and when.

I did not. Should I be doing a clean on every git pull from the scipy/scipy upstream?

No, in principle rebuilds should take care of this. However, it sometimes messes up, setuptools tracking isn't great so it could pick up outdated build artifacts. I use git clean -xdf a lot; if you don't have a fast build machine then not cleaning and only using git clean -xdf when you see weird failures  can also be a good strategy.

ugh, this was b/c I hadnt updated my virtual environment when I pulled in the new release.
conda update --all then git clean -xdf then python runtests.py -v builds fine now. Thanks @rgommers + @ilayn appreciate the help.


Configurable Polling Interval

Expose a configurable polling interval. Current is hardcoded to 10 seconds.

Implemented in v0.4.0


Enforce position of line comments

What is to be discussed?
Should we have a enforce a standard way of adding comments? Should comments on the side of lines be allowed?
// comment above
1 + 1;

1 + 1; // comment beside

Describe the solution you'd like
I usually prefer comments above its relative line/block and I catch myself very rarely wanting to add one beside code lines. Comments at the side produces longer lines and can make code difficult to read.
Additional context
Rule reference: https://eslint.org/docs/rules/line-comment-position

🤷♂


Reconsidering TEX_USE_CTEX / using XeLaTeX

I worked on manim back in 2018. I added the function for using CTeX (XeLaTeX package for Chinese) and XeLaTeX instead of LaTeX using the flag TEX_USE_CTEX in constants.py (#315).
I have stopped working on manim since 2019, but over the months there are apparently more and more people who want to use LaTeX rendering in non-English languages, and even on very old issues I still occasionally see people asking how to do that... Looking back at my change I really should have decoupled using CTeX (TeX template) from XeLaTeX (rendering tool). This has caused a lot of confusions and made weird hacks/fixes necessary for only using XeLaTeX, especially for a language that is not Chinese or English, with the most recent #858 and #840. It really should have been a flag TEX_USE_XELATEX and another flag TEMPLATE_TEX_NAME, and the flag TEX_USE_CTEX is such that when it is True, TEX_USE_XELATEX is True and TEMPLATE_TEX_NAME is "ctex_template.tex"; otherwise TEX_USE_XELATEX is False and TEMPLATE_TEX_NAME is "tex_template.tex". Then set TEMPLATE_TEX_FILE to os.path.join(os.path.dirname(os.path.realpath(__file__)), TEMPLATE_TEX_NAME). Corresponding logic: constants.py lines 74–79.
It might be even better to set it dynamically using a function or as a parameter of TexMobject(), (see issues like #891). I looked at the source code and this is definitely possible. The options I can think of are

Use the current TEX_USE_CTEX
Add flags TEX_USE_XELATEX and TEMPLATE_TEX_NAME, and rework TEX_USE_CTEX
Add parameters for TexMobject() like use_xelatex=False and tex_template="tex_template.tex"
Use the flags of 2. as a default, and make it possible to change the default using 3.

Not really sure if this is the right place to raise this issue.

I can do a PR for the change if needed

#723 is also a pretty good idea and there is a similar pr #745 although it doesn't decouple CTeX from XeLaTeX

@hanwenzhu I have tried the process as described in #840. But as soon as i set TEX_USE_CTEX to true, all my scenes show this error
Traceback (most recent call last):
File "C:\Users\Ani-Ayan PC\AppData\Local\Programs\Python\Python37\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\Users\Ani-Ayan PC\AppData\Local\Programs\Python\Python37\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Manim\manim_3_feb\manim.py", line 2, in
import manimlib.config
File "C:\Manim\manim_3_feb\manimlib_init.py", line 2, in
import manimlib.config
File "C:\Manim\manim_3_feb\manimlib\config.py", line 8, in
import manimlib.constants
File "C:\Manim\manim_3_feb\manimlib\constants.py", line 50, in
TEMPLATE_TEXT_FILE_BODY = infile.read()
File "C:\Users\Ani-Ayan PC\AppData\Local\Programs\Python\Python37\lib\encodings\cp1252.py", line 23, in decode
return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x8f in position 660: character maps to
I did not quite understand the alternative that you were suggesting. Can you be a little bit more clear as what i might try to get around this.

@Ayan2357 This was a feature request / discussion, not a solution (sorry that I didn't make this clear). Please raise a new issue for your problem (and remember to post your code and tex_template.tex or ctex_template.tex if you modified them; this will help a lot).

Closing as issue is fixed downstream at ManimCommunity. Anyone interested in multilingual support is recommended to use the community version.


Spectrally broadened pulse

Is it possible to simulate the diffraction pattern from spectrally broadened pulse

I figured it out by using convolution


Feature Request: Exposing C function sqlite3_load_extension to Java

I need to load a SQLite extension.
With sqlite-jdbc, I can enable the SQL function load_extension to achieve this. However, this is not recommended as it increases the attack surface when a SQL injection is possible.
(SQL injections should be avoided anyway but humans make mistakes ...)
SQLite provides the C function
int sqlite3_load_extension(sqlite3 *db, const char *zFile, const char *zProc,  char **pzErrMsg);
https://sqlite.org/c3ref/load_extension.html
to load extensions. This could be used to load extensions from Java but not from a SQL statement.
Loading of extensions can be enabled by C API and at the same time disabled by SQL queries.
https://www.sqlite.org/c3ref/enable_load_extension.html
https://www.sqlite.org/c3ref/c_dbconfig_enable_fkey.html (SQLITE_DBCONFIG_ENABLE_LOAD_EXTENSION)

A possible solution is to use the enable_load_extension method to disable extension loading after loading them. The downcast however isn't ideal.
import java.sql.Connection;
import java.sql.SQLException;
import org.sqlite.SQLiteConfig;
import org.sqlite.SQLiteConnection;
import org.sqlite.core.DB;

    private Connection connect() throws SQLException {
        final Connection connection = (new SQLiteConfig()).createConnection(":memory:");
        final DB db = ((SQLiteConnection)connection).getDatabase();
        try {
            db.enable_load_extension(true);
            // sql queries to load extensions
        } finally {
            db.enable_load_extension(false);
        }
        return connection;
    }


Add a pre-vm pattern to handle extf-truncf

Torch-mlir represents all scalar level values as int64 and f64. This can result in lower level failures when f64 is not supported. In most of these cases the f64 value can be completely ellided away.

I am not entirely certain how you would feel about this pattern but it seemed a reasonable fix for some torch-mlir quirks.

Move to https://github.com/llvm/torch-mlir/pull/3579


Improvements for plugin integration

@normanjaeckel Please review again.

Please also add CHANGELOG entry.

Changelog entry added. Rebased.


1.43

This question requires that you evaluate the underlined text to determine if it is correct.
Azure Key Vault is used to store secrets for Azure Active Directory (Azure AD) user accounts.
Instructions: Review the underlined text. If it makes the statement correct, select "No change is
needed". If the statement is incorrect, select the answer choice that makes the statement correct.
A. No change is needed
B. Azure Active Directory (Azure AD) administrative accounts
C. Personally Identifiable Information (PII)
D. server applications

C

A*


Seqseq example: Use symbolic loop for inference

https://twitter.com/farizrahman4u/status/914609159596859392

This is interesting, but also a lot more involved than the original version. In particular it leverages features than aren't quite private but aren't really to be found in regular use cases, like K.rnn or layer.call. Part of the goal of the example is too show that s2s in Keras is simple, and I think this looks rather complicated.
At the same time, it's good to make it available for reference. Make as a comment or something?

Couple of advantages of using symbolic loop:

Faster prediction when batch size > 1
decode_sequence method is now simpler, since we have a single "end to end" inference model.

What do you mean by make as a comment? Please elaborate.

What do you mean by make as a comment? Please elaborate.

Sorry, I meant: could we make this available as a comment? Ideally, we would add it as an alternative code snippet in the associated tutorial on the Keras blog, together with some explanations of what it does.
The primary reason for not merging it here is that it involves niche features (which I generally wouldn't recommend using other than as a workaround) and isn't as straightforward as the base version -- our goal is to make examples as simple and obvious as possible, possibly at the cost of performance.

I was trying to use the above code to eliminate the use of while loop in decode_sequence function(). But, I couldn't figure out what is "encoder" while defining the inference model.


Spurious error telemetry when stopping Windows generator tasks

Information

Onefuzz version: 3.2.0 (latest)
OS: Windows

Provide detailed reproduction steps (if any)

Create a Windows generator task, such as via the radamsa basic template.
Wait until the job's generic_generator fuzz task starts (is in running)
Stop the job with onefuzz template stop.

Expected result
The job is shut down, without any error telemetry.
Actual result
We send error-level telemetry, such as:
error running task: generate inputs failed

Caused by:
    0: generator failed to start: C:\onefuzz\ee26eeba-21b1-4da7-804c-233bd831d25b\task_tools_0\radamsa.exe
    1: The media is write protected. (os error 19)

This causes spurious integration test failures.

Using the above steps, I've been able to repro this several times in a row.

Other errors elicited by the same repro procedure (all from a single job):
generic_analysis
error running task: poller failed

Caused by:
    0: QueueClient.pop
    1: storage queue pop failed
    2: request attempt 6 failed
    3: HTTP status client error (404 Not Found) for url (<REDACTED>)

generic_crash_report
error running task: poller failed

Caused by:
    0: QueueClient.pop
    1: storage queue pop failed
    2: request attempt 6 failed
    3: HTTP status client error (404 Not Found) for url (<REDACTED>)

generic_generator
error running task: OS Error 19 (FormatMessageW() returned error 19) (os error 19) at path "C:\\Windows\\TEMP\\.tmpr3WyJK"

The template stop CLI command is used internally in the integration tests. It is invoked indirectly here, when we stop test jobs that have had their checks completed. This is set to True in the call to Run.test(), here. The inner call the stop template command is here.
The stop template command is itself defined here. It is totally non-blocking and unsynchronized. Resources are effectively all torn down at the same time. This provides the fastest user experience, but is probably the root of the spurious errors.

It turns out that the invoking onefuzz tasks delete <task_id> alone is sufficient to repro this bug, when the task is running:
error running task: generate inputs failed

Caused by:
    0: generator failed to start: C:\onefuzz\99da0cf6-3adf-4f04-85ce-2f0e6038d1e4\task_tools_0\radamsa.exe
    1: The media is write protected. (os error 19)

Using the remaining tasks from the example above, I was also able to repro just the earlier QueueClient.pop errors just via onefuzz jobs delete <job_id>.
So, it looks like we have noisy task teardown both when stopping tasks directly and when stopping tasks indirectly, via stopping the job. The template stop command is not needed to repro.

When tasks are stopped, we grab the Node that the task is running on, here. Then, we invoke Node.stop_task().
The comment on that method is outdated. We really do "send" the StopTaskNode command immediately, to support task colocation (multiple tasks sharing a VMSS node). Here, "sending" just means saving the message to a table, which eventually gets queried by the node.
But the problem is that the rest of task teardown doesn't seem to be synchronized the receipt of this command, or more importantly, the actual task state. In particular, when we call Node.stop() in Node.stop_if_complete(), we immediately call to_reimage(). However, the node supervisor only checks for commands every 10 seconds, and apparently reimaging and Azure-level teardown is often fast enough to win the race, breaking the OS-local assumptions of the not-yet-stopped task.


Creating an API section, adding Configuration API

Changes
In discussing the future of the docs, we wondered if it might be best to put the reference info into its own section so when learning about Snowpack you can get a high level view and not have to scroll through a bunch of options.
Goals

Standardize format and think about what format might look to if we ever moved to automated docs. For example OpenAPI or mson. Some inspiration taken from React and Webpack API docs.
Separate the API reference from other main sections meant to be read rather than used as reference

What to review

I wrote "summary" sections for each major section that didn't have one, some of them are pretty...bare bones like config.buildOptions : Determines how Snowpack processes the final build.
structure and formatting
the type: properties additions to #### level items

Future

The configuration section will be rewritten to give a better high level view (why am I doing configuration? what are some examples)
Possibly integrate the API docs into the code itself so we could automate some of this

Testing
Someday docs will have tests, but today is not that day.
Docs
This is documentation

two smaller comments:

add a "plugin API" section to the "API Reference" section, that is really and just links to /plugins page
in header, change "config" to "API Reference" + change link to /#api-reference (currently links to /#configuration)


About FFHQ 256*256 generator

Could you tell me how can you get the 256×256 generator of FFHQ? Did you get from official pretrained model or trained on ffhq dataset(resized to 256×256) from scratch?

We train a new model on FFHQ images (with 256 resolution) from scratch.

Thank you!


feat(cli): add dev-contracts

Adds a dev-contracts script to the CLI to:

start a dev node in the background (or use one provided via the --rpc arg)
watch config and file changes, run all necessary codegen steps and redeploy contracts when stuff changes

does this impact our need for initialize? been kinda unhappy with that pattern and I hope its replaced by all of this

does this impact our need for initialize? been kinda unhappy with that pattern and I hope its replaced by all of this
there is still some bare-minimum to do before the client starts up so we don't get client import errors (e.g. world types)

Yep, at least for dev there is no more need for initialize! Still kept the script around because it's still needed for deploy for now if you didn't run dev before

one thing we could do in the future is split dev-contracts with a 1) deploy prerequisites and 2) deploy, then pnpm dev runs prerequisites then runs client + contracts dev (but with a flag in contracts dev to skip prereqs)

can you add docs to the cli page? with the new command + args

actually, on the first ever run of dev it might be that the client doesn't have the world types yet. Wonder how we can get around that without having to run initialize before every start...

Fixed this by running the codegen step once before anvil starts, and making the clients wait for anvil before starting their dev server


Read the docs

[x] Sign up for RTD
[x] Setup python code to generate docs while building the package
[x] Setup automated pipeline github actions

Github pages were added and RTD documentation was generated on a different branch.


arrow-cpp: 6.0.0 -> 6.0.1

Motivation for this change
Bump arrow-cpp from 6.0.0 to 6.0.1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Apache Arrow 6.0.1 (2021-11-10)
Bug Fixes

ARROW-14437 - [Python] CSV test_cancellation unittests fail on Apple M1
ARROW-14492 - [JS] Browser bundles broken
ARROW-14513 - [Release][Go] Update release-6.0.0 with /v6 suffix
ARROW-14519 - [C++] joins segfault when data contains list column
ARROW-14523 - [C++][Python] S3FileSystem write_table can lose data
ARROW-14538 - [R] Work around empty tr call on Solaris
ARROW-14550 - [C++][Go] the json license is not compatible with FSF standards
ARROW-14583 - [R][C++] Crash when summarizing after filtering to no rows on partitioned data
ARROW-14584 - [Python][CI] Python sdist installation fails with latest setuptools 58.5
ARROW-14620 - [Python] Missing bindings for existing_data_behavior makes it impossible to maintain old behavior
ARROW-14630 - [C++] DCHECK in GroupByNode when error encountered

New Features and Improvements

ARROW-13156 - [R] bindings for str_count
ARROW-14181 - [C++][Compute] Hash Join support for dictionary
ARROW-14189 - [Docs] Add version dropdown to the sphinx docs
ARROW-14310 - [R] Make expect_dplyr_equal() more intuitive
ARROW-14365 - [R] Update README example to reflect new capabilities
ARROW-14390 - [Packaging][Ubuntu] Add support for Ubuntu 21.10
ARROW-14433 - [Release][APT] Skip arm64 Ubuntu 21.04 verification
ARROW-14450 - [R] Old macos build error
ARROW-14459 - [Doc] Update the pinned sphinx version
ARROW-14480 - [R] Expose arrow::dataset::ExistingDataBehavior to R
ARROW-14486 - [Packaging][deb] libthrift-dev dependency is missing
ARROW-14490 - [Doc] Regenerate CHANGELOG.md to include all versions
ARROW-14496 - [Docs] Ensure links to non-sphinx parts of the docs are relative instead of absolute
ARROW-14499 - [Docs] Version dropdown side-by-side with search box
ARROW-14514 - [C++][R] UBSAN error on round kernel
ARROW-14580 - [Python] PyPI listing does not include trove classifiers for Python 3.10
ARROW-14623 - [Packaging][Java] Upload not only .jar but also .pom
ARROW-14628 - [Release][Python] Use python -m pytest

@GrahamcOfBorg build python38Packages.pyarrow python39Packages.pyarrow

@veprbl I think the issue is jemalloc on aarch64-darwin. Not sure why though. Unfortunately I don't have access to a machine to test this on.

@veprbl I think the issue is jemalloc on aarch64-darwin. Not sure why though. Unfortunately I don't have access to a machine to test this on.

It's already failing on Hydra https://hydra.nixos.org/build/158150169 . I would leave it be for now – it may resolve by itself.

Result of nixpkgs-review pr 146823 run on x86_64-darwin 1

2 packages marked as broken and skipped:

python38Packages.spacy-transformers
python39Packages.spacy-transformers

6 packages failed to build:

python38Packages.intake
python38Packages.tokenizers
python38Packages.transformers
python39Packages.intake
python39Packages.tokenizers
python39Packages.transformers

13 packages built:

arrow-cpp
python38Packages.datasets
python38Packages.google-cloud-bigquery
python38Packages.intake-parquet
python38Packages.pyarrow
python38Packages.snowflake-connector-python
python38Packages.snowflake-sqlalchemy
python39Packages.datasets
python39Packages.google-cloud-bigquery
python39Packages.intake-parquet
python39Packages.pyarrow
python39Packages.snowflake-connector-python
python39Packages.snowflake-sqlalchemy

@veprbl Thanks for fixing. Now that parquet-tools is merged, I'll probably need to adjust the tests. Looking into that now.

@veprbl Is this ready to merge?

Result of nixpkgs-review pr 146823 run on x86_64-darwin 1

2 packages marked as broken and skipped:

python38Packages.spacy-transformers
python39Packages.spacy-transformers

7 packages failed to build:

python38Packages.google-cloud-bigquery
python38Packages.intake
python38Packages.tokenizers
python38Packages.transformers
python39Packages.intake
python39Packages.tokenizers
python39Packages.transformers

13 packages built:

arrow-cpp (parquet-cpp)
parquet-tools
python38Packages.datasets
python38Packages.intake-parquet
python38Packages.pyarrow
python38Packages.snowflake-connector-python
python38Packages.snowflake-sqlalchemy
python39Packages.datasets
python39Packages.google-cloud-bigquery
python39Packages.intake-parquet
python39Packages.pyarrow
python39Packages.snowflake-connector-python
python39Packages.snowflake-sqlalchemy


chore: loader bar added

https://github.com/fossasia/open-event-next/issues/16

You've undone the styling of the project

You've undone the styling of the project

@iamareebjamal okay let me do.

Not what I meant by that

then?

as their are some conflicts i am creating different pr.


Rename foster_application_reviews to manage_fosters

We currently have a route, controller, and views for foster_application_reviews. This naming is not accurate and needs to be changed to manage_fosters
Acceptance Criteria:

change route, controller, view
update tests so they pass

Can you assign this to me?

Done @LindseyB


Minor language cleanup in ROADMAP.md

No intent was harmed by this change (I hope).

Thanks Maru. These texts are blatantly copied from https://github.com/docker/docker/issues/9983 :) with the intention of not changing the meaning.
But, your changes LGTM for merging.

LGTM too

LGTM


Applying filter / sort / etc.. on a CollectionProperty returns a Stream of CollectionChangeset

Hi,
In version 2.0 if you apply for example a sort operation on a CollectionProperty<T> then the result is a Stream<CollectionChangeset<[T]>>
In version 1.x if we would apply a sort operation on a ObservableCollection<T> we would again get a ObservableCollection<T>.
Can you give me some background information about this change (if this was intended).

To give a bit more information. Currently I resolve this with the code example below, but not sure if this is the correct way.
let unsortedCollection = CollectionProperty<[String]>([])
let sortedCollection = CollectionProperty<[String]>([])

unsortedCollection.sort{$0 < $1}.observeNext{ changeset in
       sortedCollection.update(changeset)
}

It was bad design in v1. Because CollectionProperty is not a value type, you could have easily gotten into an unexpected state. Mapped ObservableCollection had a link to the parent ObservableCollection, but could have also been modified manually.
Now mapping / filtering / sorting operates only on event stream, just like in case of plain Property, so unexpected state cannot happen.
This is a bit restrictive, but old behaviour can be simulated just as you found ot. In beta4 I made CollectionProperty bindable so it's a bit simpler to do that what you're doing:
let unsortedCollection = CollectionProperty<[String]>([])
let sortedCollection = CollectionProperty<[String]>([])

unsortedCollection.sort{$0 < $1}.bindTo(sortedCollection)

Great makes sense, appreciate the update in beta4. Will close the issues.


When a time is displayed, it should be done so uniformly across the app

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Submission validity

[X] This is not a personal support request, that should be posted on discussions community.
[X] I checked current issues and this request isn't a duplicate of an existing issue, opened or closed.

Summary
When a time and date is displayed in YOURLS's admin pages, it isn't done so uniformly.
E.g. within index.php's list of URLs, it displays Nov 19, 2021 16:33 (24 hour)
Where as on the graph displaying the time of visitors it uses the format HH AM/PM, e.g. 13 PM. (mixing 24 hour & 12 hour formats)
Under 'Historical click count' for a specific URL's stats, there is Short URL created on November 19, 2021 @ 4:33 pm. (12 hour)
The display of the date appears to be consistent, as with the examples above.
Example of where two times are displayed different on a single page:

Motivation
Making the display of the time uniform across the system assists with being able to compare times quickly and more easily.
Whilst there may be a myriad of different ways to display the time (especially in different cultures), the attention to detail to display time in a single fashion helps the user know that time and care has been taken to create the product.
This should also help users whose first language isn't English to read the times more consistently.
Describe the solution
Choose a single format to display times across the system.
I believe the main options should be:
HH:MM (24 hours)
HH:MM AM/PM (12 hours)
These two formats are more universally accepted than HH:MM a/p (12 hours), which is sometimes favoured in the USA.
I don't believe the first solution is allowing the user to choose the format they would like the time/date to be displayed in, as this would require significant work. The WordPress system does allow a custom time/date format displayed.

This may be merged/related to improving the stats page, #2675.

The reasoning (not stating that it was a good idea, just explaining why I did this :) of various time/date formatting was primarily a matter of "where do I need a compact display" and "where can I afford to be more verbose". I think. Also maybe a complete lack of consistency.
Did you give https://github.com/YOURLS/timezones a try ? I think this is what you're looking for.

Thanks @ozh for response. I really appreciate it.
I was not aware of the plugin, so I've added that in and it has certainly helped.
It's still inconsistent on the graph, but I suspect it's got a library of its own and decides to display the time however it wishes.
The whole thing was a recommendation/thought rather than a demand, so hopefully it came across that way.
Again, thank you for the response and more broadly for contributing YOURLS to the open source community!


Compatibility with raspberry pi camera module 3?

Hi.
I just bought a 12 MP raspberry pi camera module 3 and have connected it with a Pi zero 2W. I have tried using the cam web interface with the camera module 3 but it gives various errors. Can you kindly tell me whether it works with camera module 3?
Thank you.

This software uses the GPU legacy camera support which only supports v1 and v2 module rapsberry cameras.

This software uses the GPU legacy camera support which only supports v1 and v2 module rapsberry cameras.

Okay. Thank you for responding.

No workaround to pipe video from camera v3 to this?

Hi @biprav, you might be interested in this. This program should be usable on the pi camera module 3 as it uses Picamera2 (which is developed by Raspberry Pi themselves) as a foundation. Be aware that not all functionality has been implemented but it should be a good starting point ^_^


这是由于显示窗口较小，出现了多行显示的问题，可以把显示窗口调大一点，这样训练进度条会显示在一行中。


Support looking up meta-data by build id

Currently the agent supports looking up meta-data by job, but this can be awkward in triggered builds where you are trying to lookup meta-data on the original build.
buildkite-agent meta-data get --build 07cfd694-0c0f-4dd1-bc0a-bfbe0327222
This would require backend changes (implementation of /build/{build-id}/data/get), PR for discussion.

@buildkite-agent-releaser I would love to see this get added. We’re waiting on this exact future to unblock some quick and easy pipeline to pipeline communication. Thank you!

Looks like this has been implemented in https://github.com/buildkite/agent/pull/2025 and released in v3.46.0.
Thanks for the suggestion!


Doesn't work with the new Mongoose 5.x RCs

One difference I've already noticed is some errors are returned as BulkWriteError and not MongoError.

@matteodelabre can you merge this bad boy?

@johnculviner +1
@matteodelabre do you still support your plugin?

Thanks everyone for your patience. I have merged David’s contribution and will publish a new version soon after some tweaking and further testing.

Thanks for the help (and the library!)


Update README.md

Update the TIPs section.

Don't merge this until the following PRs have been merged:

https://github.com/paketo-community/cargo/pull/146
https://github.com/paketo-community/rustup/pull/45

And a new release has been cut.


SOFT Cache is not working properly when batch inserting

Some records are discarded when saving many values in one commit while soft cache is enabled. There is no problem with cache disabled or MRU cache. In my configuration, transaction are disabled, i don't know if problem occurs when transaction are enabled.
This problem doesn't occur in JDBM2.

Fixed in recent Alpha2.
Thanks for reporting all this bugs, it really helps.


HttpStatusBasedRetryStrategy should allow retrying on exceptions rather than just HTTP status

Currently, HttpStatusBasedRetryStrategy ignores exceptions in the result. It is common to have exceptions like ClosedSessionException, which should be retried since they can be random socket closures, so it seems like a bad default to ignore the exceptions. Should we have it default to retrying on exception?

Sorry for the late response.
As you said, it will be better retrying when those exceptions happen by default. I will make a PR for that. Thank you~!!

I think it highly depends on how and when such an exception has been raised. That being said, we have some questions like:

Should we retry on any Exception types or on just a few specific types?
Should we retry when the HEADERS frame of the response has been received but connection has failed (or exception has been raised) before the last frame is received?
Should we retry when an error occurred while preparing or sending a request?

I think this highly depends on what HTTP API we are calling. e.g. for GET, we will want to retry aggressively while we may not (or may depending on implementation?) for non-idempotent methods.

I agree that it shouldn't be all-encompassing, and questionable to default, but it is probably useful to add knobs on the builder we already have to make it simple to treat exception as failure. Otherwise, the only option is forking lots of logic like this
https://github.com/curioswitch/curiostack/blob/master/common/google-cloud/armeria-cloud-storage/src/main/java/org/curioswitch/curiostack/gcloud/storage/ErrorRetryStrategy.java

Agreed. How about this:

Provide a builder like RetryStrategyBuilder which provides various knobs, HttpStatus and Exception most notably.

Do some composition, like composing HttpStatusBasedRetryStrategy and ExceptionBasedRetryStrategy?

Change RetryStrategy.on*Status() to default to 'retry on any exceptions thrown before content preview is done.'

Closed by #1056. Thanks @anuraaga and @minwoox for suggestion and fix.


Azure AD Connect Health for AD FS support for Server 2019?

Is Server 2019 ADFS supported? aka v5?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 79ab2bbb-4adc-3106-c8dc-3462cc23c587
Version Independent ID: 8fa8f32e-ceca-6604-f2e3-a994efd17ecc
Content: What is Azure AD Connect and Connect Health.
Content Source: articles/active-directory/hybrid/whatis-azure-ad-connect.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@googol13 Thanks for your question. We are checking on this and will respond to you soon.

@googol13 it is not yet supported on Server 2019 but will be in the future.

They are working on it and I would check the release notes for updates on this.
https://docs.microsoft.com/en-us/azure/active-directory/hybrid/reference-connect-version-history
https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/overview/whats-new-active-directory-federation-services-windows-server


Command sf apex run test fails randomly with error "Unknown error in Apex Library: Error authenticating with JWT"

Summary
An error occur randomly when running the command sf apex run test --test-level RunLocalTests on a Scratch org : Unknown error in Apex Library: Error authenticating with JWT.
This seems to happen in 25% of executions.
See error below :

And sometimes (75%), the execution is successful :

Steps To Reproduce
Private repo, sorry.
Expected result
The Apex tests are executed without loosing the auth. to the Scratch Org.
Actual result
Sometimes, an error occur : Unknown error in Apex Library: Error authenticating with JWT.
System Information

{
  "architecture": "win32-x64",
  "cliVersion"<EMAIL_ADDRESS>  "nodeVersion": "node-v20.11.1",
  "osVersion": "Windows_NT 10.0.22621",
  "rootPath": "C:\\Users\\corentin.romilly\\AppData\\Roaming\\npm\\node_modules\\@salesforce\\cli",
  "shell": "cmd.exe",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 3.0.18 (core)",
    "@oclif/plugin-commands 3.3.4 (core)",
    "@oclif/plugin-help 6.0.22 (core)",
    "@oclif/plugin-not-found 3.1.9 (core)",
    "@oclif/plugin-plugins 5.0.21 (core)",
    "@oclif/plugin-search 1.0.24 (core)",
    "@oclif/plugin-update 4.2.12 (core)",
    "@oclif/plugin-version 2.1.2 (core)",
    "@oclif/plugin-warn-if-update-available 3.0.19 (core)",
    "@oclif/plugin-which 3.1.10 (core)",
    "@salesforce/cli 2.42.6 (core)",
    "apex 3.1.11 (core)",
    "auth 3.6.11 (core)",
    "data 3.3.5 (core)",
    "deploy-retrieve 3.6.14 (core)",
    "info 3.2.8 (core)",
    "limits 3.3.7 (core)",
    "marketplace 1.2.5 (core)",
    "org 4.1.10 (core)",
    "packaging 2.4.4 (core)",
    "schema 3.3.6 (core)",
    "settings 2.2.6 (core)",
    "signups 2.0.28 (user) published 87 days ago (Sun Mar 03 2024) (latest is 2.3.5)",
    "sobject 1.4.2 (core)",
    "source 3.3.12 (core)",
    "telemetry 3.3.7 (core)",
    "templates 56.2.7 (core)",
    "trust 3.6.11 (core)",
    "user 3.5.8 (core)"
  ]
}

Additional information

same here

I have the same issue

a few questions:

Are these failures more frequent on CI where new scratch orgs are created? the screenshot looks like it's from CI.
if so, how do you authenticate to scratch orgs?

related:
https://github.com/forcedotcom/cli/issues/1640#issuecomment-1286961544
the comment from above refers to this failure when trying to auth to a new scratch org using its creds, but this same flow happens when refreshing an expired access token.
Also, can you provide enable debug logs (pass --dev-debug flag) and share the results from the failure?


Typo "team explorer"→"Team Explorer"

https://docs.microsoft.com/en-us/azure/devops/repos/tfvc/get-code-reviewed-vs?view=azure-devops
https://github.com/MicrosoftDocs/azure-devops-docs/blob/main/docs/repos/tfvc/get-code-reviewed-vs.md
#PingMSFTDocs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off

@steved0x: I'm sorry - only the author of this article, @MaximMalygin, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


What would you like us to improve?
Target audience: Is the documentation tailored for developers with different levels of experience (beginners, intermediate, advanced)?
Terminology: Are technical terms clearly defined and explained for those unfamiliar with them?
Structure and organization: Is the information logically organized and easy to navigate, with clear headings and subheadings?
Completeness: Does the documentation cover all essential features and functionalities, or are there gaps?
Examples and use cases: Are there sufficient code examples and practical use cases to illustrate concepts?
Visual aids: Are diagrams, screenshots, or other visual aids used effectively to enhance understanding?
Error messages and troubleshooting: Are error messages clear and actionable, and is there adequate troubleshooting guidance?
Community resources: Does the documentation point to relevant community forums, support channels, or additional resources?
How important it is?
For individual users:
Frustration and wasted time: Users struggling to understand the documentation may become frustrated and abandon their attempts to use Scroll, hindering their participation in the ecosystem.
Errors and security risks: Misinterpreting documentation can lead to errors in using Scroll, potentially impacting their funds or assets. Unclear security instructions could leave users vulnerable to attacks.
Learning curve and adoption: Poorly explained concepts can create a steeper learning curve, making it harder for new users to onboard and contribute to the ecosystem.
For the Scroll community:
Negative reputation: Unclear documentation can reflect poorly on the Scroll project, hindering its image as a user-friendly and accessible platform.
Limited ecosystem growth: Difficulty using Scroll due to unclear documentation can slow down the adoption of new protocols and applications within the ecosystem.
Support burden: If users repeatedly encounter confusion due to documentation, this can increase the support burden on the Scroll team, diverting resources from other development efforts.
Additionally:
Accessibility and inclusivity: Clear documentation is crucial for ensuring the inclusivity of the Scroll community, catering to users with different levels of technical expertise and backgrounds.
Building trust and confidence: Well-written documentation shows attention to detail and professionalism, contributing to user trust and confidence in the Scroll platform and its capabilities.

We understand the importance of knowing your audience. If you have specific feedback on the documentation, a section or a specific article, please provide it in new issues.


TensorBoard maybe timeout when show imagenet inception-3 event

when i run imagenet example ; in tensorflow/models/image/imagenet/classify_image.py i add

w=tf.python.training.summary_io.SummaryWriter( sess.graph_def, FLAGS.model_dir, 'graph.pbtxt')
tf.train.SummaryWriter.flush(w)
tf.train.SummaryWriter.close(w)

output a file: events.out.tfevents.1452154974.ip-172-30-.-.
when i open tensorboard , in tab  graph ,then i check top,tensorboard use cpu 100%; but wait a while ,tensorboard cpu use is down; tensorboard log a http 200:"GET /graph?run=%2Fopt%2Fpractice%2Ftf%2Fimagenet%2Flog%2F HTTP/1.1" 200 ;
maybe event file so big cause http timeout??
92M Jan  7 09:01 log/events.out.tfevents.1452154974.ip-172-30
any one can help?

i will check cifar-10 example for what missing?

@zdx3578 so the graph page displays nothing?

@danmane: Do you know if this is still an issue?

This looks like a duplicate of #1287 , which is fixed as of latest master. @zdx3578 if the fix for #1287 doesn't work for you, feel free to re-open and we'll dig into it a bit more - if you do re-open it, can you please upload an events file for which you aren't able to visualize the graph? That will make it a lot easier to dig into.


Ruby Tools for Visual Studio

I was wondering if anyone was wanting Microsoft to bring Ruby to Visual Studio?
I have a good start.
Thanks,
Jonathan

Hi @jdm7dv
You should post this feature idea at https://developercommunity.visualstudio.com/
Click on Suggest a feature
(this is the replacement for the uservoice site)
You'll get more of a general audience there and be able to see vote counts.
Thanks


Collapse example code blocks in challenge descriptions

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

We added a dropdown toggle button to all example code blocks on the https://www.freecodecamp.org/learn courses. This aims to make dealing with large code examples less awkward for the user. Here is an example:
Toggled on (default state):

Toggled off:

Closes #54190

Do we want this to be applying itself to ALL code blocks? Including those that aren't in the multifile layout?

That makes sense yes


Spyder 5 can not be updated - Dependencies issue?

Issue report checklist.pdf

Hey @jeng87, thanks for reporting. That is not a dependencies problem, it's due to the fact that Anaconda has not updated packages for our latest version yet.
Until that happens, you have two options:

Use our Windows installer, which you can find here.

Create a new environment with conda-forge packages. For that, please close Spyder, open your terminal and run the following commands there:
 conda create -n spyder-cf -c conda-forge spyder
 conda activate spyder-cf
 spyder

After that, you can connect Spyder to your base environment by following these instructions:
http://docs.spyder-ide.org/current/faq.html#using-existing-environment

Hi, Thank you for the help. I have made new environment with conda-forge packages for spyder, and I follow the step to connect spyder to the environment that I need to work with. But another issue pop up like below showing in the spyder console. And I install spyder-kernels 2.2, but still not solving the problem. Can you please help me with this new issue. Thank you.
The Python environment or installation whose interpreter is located at
/home/lorizeltser/anaconda3/envs/ClearMapStable/bin/python
doesn't have the spyder‑kernels module or the right version of it installed (>= 2.2.1 and < 2.3.0). Without this module is not possible for Spyder to create a console for you.
You can install it by activating your environment (if necessary) and then running in a system terminal:
conda install spyder‑kernels=2.2
or
pip install spyder‑kernels==2.2.*

And I don't understand. I installed anaconda in Nov last year, which should be their newest version. Why I would have package out data problem. Thank you.

Hey @jeng87, you said:

But another issue pop up like below showing in the spyder console

To solve this problem, you also need to install spyder-kernels from conda-forge, like this
conda install -c conda-forge spyder-kernels

If you get errors in your current environment after doing that, please create a new environment with spyder-kernels and the packages you need to use (e.g. Scikit-learn):
conda create -n myenv -c conda-forge spyder-kernels scikit-learn

And I don't understand. I installed anaconda in Nov last year, which should be their newest version. Why I would have package out data problem. Thank you.

I know this is very confusing, but the problem is Anaconda takes a lot of time to update Spyder. For instance, we released our 5.1.5 version six months ago, but that's the last one available in Anaconda. Whereas our latest version (5.2.2) was released a month ago.
This has become the norm in Anaconda (before they took at most a week to upload packages for a new version), which is why we now recommend our users to use Conda-forge instead.
Let us know if there's something else we can help you with.

jeng87.zip
Hi @ccordoba12 I upload the environment file in here. And the spyder-kernels error still exist. If you need to run the script I am using. I can upload them, but it will take a while because I have to upload lots of raw data. thank you.

https://christophkirst.github.io/ClearMap2Documentation/html/installation.html#cloning
@ccordoba12 Here is the pipeline of installation of the tool I am using, and seems like the environment file is the frozen package versions

Hi @ccordoba12,
I have fix this problem by uninstalling an alternative environment file first with flexible package dependencies. The previous environment is created with all frozen package dependencies, and thus it seems I couldn't do anything at that time when I was trying to use the newest Spyder. Thank you so much helping me with this issue.

Great! I'm really glad you were able to solve your problem. And thanks for letting us know about it.


Improve a11y for template counter component

Before submitting the PR, please make sure you do the following

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts

This PR includes the following a11y improvements for the default template:

The Counter component renders the next number out-of-view so it can slide the next number in. This number was still visible to assistive tech such as screen readers, which read out both numbers as if they were one. For example, NVDA read out "10" in the starting state, when only "0" was visible on screen. Adding the aria-hidden attribute to the next number hides it from screen readers so they only read out the visible number.
I also added aria-hidden to the SVGs in the + and - buttons. The aria-label on the buttons overrides the label on the SVGs, and they don't need to be read out to screen readers.

thank you so much!